home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part1 / 955 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  3.0 KB
  1. Path: news.iag.net!news
  2. From: jatmon@iag.net (John R Buchan)
  3. Newsgroups: comp.lang.c
  4. Subject: Re: allocating unlimited string input....HELP
  5. Date: 10 Jan 1996 14:38:19 GMT
  6. Organization: Internet Access Group, Orlando, Florida
  7. Message-ID: <4d0j0r$7aq@news.iag.net>
  8. References: <4cvph6$s8s@pulp.ucs.ualberta.ca>
  9. NNTP-Posting-Host: pm1-orl23.iag.net
  10. X-Newsreader: WinVN 0.99.7
  11.  
  12. In article <4cvph6$s8s@pulp.ucs.ualberta.ca>, ryangall@gpu.srv.ualberta.ca 
  13. says...
  14. >
  15. >I am doing an assignment for my computing class, and seem to have run into 
  16. >a small problem. I am supposed to be able to read an unlimited string from 
  17. >stdin, without defining maximum size. I thought that this problem would be 
  18. >simple....but I have been at it for a while now. Ill show you what my last 
  19. >resort was.....
  20. >
  21. >
  22. > 
  23. >int read(char *S)
  24. >{
  25. > char *A,*B;
  26. > char ch,count=0;
  27. >
  28. >  A=NULL;
  29. >  B=NULL;
  30. >
  31. >  ch=getc(stdin);
  32. >
  33. >  while(ch!='\n')
  34. >  {
  35. >    B=(char *) malloc(count+2);
  36. >    if(count>0)
  37. >    { 
  38. >      sprintf(B,"%s%c",A,ch);
  39. >      free(A);
  40. >    }
  41. >    else sprintf(B,"%c",ch);
  42. >
  43. >    printf("%s---%i----%i\n",B,strlen(B),count);
  44. >    A=B;
  45. >    ch=getc(stdin);
  46. >    count=count+1;
  47. >  }
  48. >
  49. >}
  50. >
  51. <snip>
  52. >Anyways, the function reads in just fine, but for some reason when it gets 
  53. >to 146 characters it bails out, loses char *B and starts reducing 
  54.  
  55. Are you sure it doesn't start acting up, when count reaches 128?  That's 
  56. the result where char is a signed char.  At 127 + 1, count goes negative 
  57. (becomes -128).  Subsequent loops would count up from there.
  58.  
  59. Of course, the first malloc call with a negative size causes a failure, but 
  60. your code isn't testing for allocation failure.  So you don't catch it
  61. there.  Instead you start using invalid pointers in your sprintf and free 
  62. calls.  From that point on anything can happen.
  63.  
  64. You should probably see count go negative in the output from your printf.  
  65. Unfortunately, your format string used '-' between the fields, so the 
  66. negative sign simply blends in.
  67.  
  68. So you need to:
  69.  
  70.    1. define count as a larger type.
  71.    2. add a condition to your loop to ensure that count doesn't exceed
  72.       the maximum for its type (look in limits.h for macros).
  73.    3. add an allocation failure test (if(B==NULL) {/* handle fail*/}).
  74.    4. modify the format string in your printf.
  75.    5. define ch as int (the correct return type for getc).
  76.  
  77. You might want to:
  78.    1. loop into the realloc function.  It is made for this type of code.
  79.    2. d/l and read the c.l.c faq (Frequently Asked Question) list.  It is 
  80.       available for anonymous ftp from rtfm.mit.edu /pub/usenet/comp.lang.c.
  81.    3. remove the cast from the malloc return.  It is not necessary in ansi
  82.       c and can hide some errors.
  83.    4. add ch == EOF as a break condition for your loop.
  84.  
  85. >please mail me ASAP....thanks!
  86.  
  87. Sorry, I don't think I have a large enough envelope and I certainly can't 
  88. afford the postage.  However, I will email a copy of my reply to you :-)
  89.  
  90. -- 
  91. John R Buchan           -:|:-     Looking for that elusive FAQ?  ftp to:
  92. jatmon@mail.iag.net     -:|:-     rtfm.mit.edu /pub/usenet-by-group/....
  93.  
  94.